Skip to content

Conversation

@Mic92
Copy link
Member

@Mic92 Mic92 commented Jul 16, 2022

No description provided.

@Mic92 Mic92 merged commit 35c1934 into master Jul 16, 2022
@Mic92 Mic92 deleted the compiler-check branch July 16, 2022 06:12
@mohammad-akhlaghi
Copy link

Thank you very much for adding the check!

Please note that many older systems (where users don't have root permissions) don't yet have such recent compilers. It was on one such system that I found the bug above! I had to go back to patchelf 0.9 to be able to install it there!

PatchELF is a very low-level program, and we use it in Maneage to build low-level components (like the compiler!). But if we can't use PatchELF on systems with an old compiler, then we won't be able to build a more recent compiler afterwards!

Given its low-level and fundamental nature, be usable on systems with older compilers, it would be great if you can keep PatchELF compatible with at least the C++ standard of ~20 years ago (for example C++ of 2003 in this case).

@Mic92
Copy link
Member Author

Mic92 commented Oct 28, 2022

Sorry, I am not interested in using supporting older c++ standards. You can:

  • Either use the statically compiled executable provided by the project
  • Cross compile patchelf on a system with a newer compiler
  • Use an older version of patchelf that is compatible your compiler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants